Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added read aloud #144

Merged
merged 2 commits into from
Jun 10, 2024
Merged

Conversation

rashiranjan22
Copy link
Contributor

Related Issue

#121

Description

added read aloud option so that users can listen to the articles

Type of PR

  • Feature enhancement

Screenshots / videos (if applicable)

Media1.mp4

Checklist:

  • [ X I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Additional context:

I also fixed base.html path specified in index.html and bookmarks.html

@rashiranjan22
Copy link
Contributor Author

@SiddharthBahuguna , please check

@SiddharthBahuguna
Copy link
Owner

@anurag6569201 Please check.

@rashiranjan22
Copy link
Contributor Author

@SiddharthBahuguna , is something missing ??

@anurag6569201
Copy link
Collaborator

@rashiranjan22 what us the use of start stop pause

@rashiranjan22
Copy link
Contributor Author

rashiranjan22 commented Jun 10, 2024

when one click on start button it fetches the article content on the website the url directs to and start reading it
with pause/resume one can pause it and resume it where it was left
stop button stops it completely , then one cannot resume it , it would start all over again

@anurag6569201 anurag6569201 merged commit fa50c56 into SiddharthBahuguna:main Jun 10, 2024
1 check passed
@rashiranjan22
Copy link
Contributor Author

@SiddharthBahuguna , please label the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants